Skip to content

chore: refactor faro (CE)#40350

Merged
ApekshaBhosale merged 7 commits intoreleasefrom
chore/refactor-faro
Apr 29, 2025
Merged

chore: refactor faro (CE)#40350
ApekshaBhosale merged 7 commits intoreleasefrom
chore/refactor-faro

Conversation

@dvj1988
Copy link

@dvj1988 dvj1988 commented Apr 23, 2025

Description

Tip

Add a TL;DR when the description is longer than 500 words or extremely technical (helps the content, marketing, and DevRel team).

Please also include relevant motivation and context. List any dependencies that are required for this change. Add links to Notion, Figma or any other documents that might be relevant to the PR.

Fixes #Issue Number
or
Fixes Issue URL

Warning

If no issue exists, please create an issue first, and check with the maintainers if the issue is valid.

Automation

/ok-to-test tags="@tag.All"

🔍 Cypress test results

Tip

🟢 🟢 🟢 All cypress tests have passed! 🎉 🎉 🎉
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/14664226456
Commit: caaee4b
Cypress dashboard.
Tags: @tag.All
Spec:


Fri, 25 Apr 2025 13:20:56 UTC

Communication

Should the DevRel and Marketing teams inform users about this change?

  • Yes
  • No

Summary by CodeRabbit

  • Refactor

    • Unified all error and telemetry reporting to use a centralized telemetry utility for improved consistency.
    • Replaced legacy error reporting imports and methods with a new singleton telemetry interface across the application.
    • Removed obsolete telemetry and error reporting files and classes.
    • Simplified telemetry initialization by removing conditional tracing checks.
    • No changes to user-facing functionality or workflows.
  • Tests

    • Updated test mocks to use the new telemetry interface.

Loading
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ok-to-test Required label for CI skip-changelog Adding this label to a PR prevents it from being listed in the changelog

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants